Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: create tags for new revisions published to the store #23

Merged
merged 2 commits into from
Jan 3, 2024
Merged

feat: create tags for new revisions published to the store #23

merged 2 commits into from
Jan 3, 2024

Conversation

jnsgruk
Copy link
Member

@jnsgruk jnsgruk commented Dec 15, 2023

Hey 👋

So as discussed in our catch-up meet, here is a first draft of how we might want to create tags automatically in repositories based on the revisions that were released to the store. The aim here is to make debugging issues a little easier, as when an issue is reported on a snap with a given revision, you can just checkout the tag and build/install/troubleshoot.

It's a little complicated with multiple architectures. The design I've chosen is to create one tag per revision, and name with the arch just to make searching/bisecting easier. The tag names will be rev<num>/<arch>.

You can see an example run here and the resulting tags here.

I should note that this is actually a breaking change because all repos that use these workflows will need to add a new input to the release workflow. I think if we merge this, we should tag the repo at v1, and work on using the actions with a versioned reference like @v1 rather than @main.

If this does merge, I'll do some follow-up PRs to address the incompatibilities on the other repos.

@jnsgruk
Copy link
Member Author

jnsgruk commented Dec 15, 2023

A note that one thing we could consider is including the version of the upstream project in the tag, e.g. for Signal we could have something like 6.42.0/rev576/amd64 - happy to iterate on the format.

@merlijn-sebrechts
Copy link
Member

+1 for adding the snap version to the tag, this will make it more clear, imo. I'll take a better look at this PR later.

@jnsgruk
Copy link
Member Author

jnsgruk commented Jan 2, 2024

@merlijn-sebrechts version added to tag name in 78edeec (#23)

Copy link

@popey popey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice update!

@jnsgruk jnsgruk merged commit 64948df into snapcrafters:main Jan 3, 2024
1 check passed
@jnsgruk jnsgruk deleted the tag-revisions branch January 3, 2024 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants